Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ports the RBMK from NSV (for real this time) #10364

Merged
merged 51 commits into from
Nov 23, 2024

Conversation

XeonMations
Copy link
Contributor

@XeonMations XeonMations commented Dec 20, 2023

About The Pull Request

Adds in the RBMK nuclear reactor from NSV. (this time hopefully without pull breaking map changes)

All things are able to be ordered from cargo.

Wiki can be found here: https://wiki.beestation13.com/view/Guide_to_the_Nuclear_Reactor

(I accidentally created the wiki on the main wiki instead of a sandbox, I'm sorry)

RBMK Hum noise sourced from here, under CC BY 4.0 and edited by me.

Why It's Good For The Game

More content for engineering!

Testing Photographs and Procedure

Screenshots&Videos

image
image
image
image
image
image

Changelog

🆑 XeonMations, Aramix, DungeonVR
add: Added the RBMK reactor, fuel rods and engine sealant, all orderable from cargo!
/:cl:

@github-actions github-actions bot added Sprites Sound TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Dec 20, 2023
@XeonMations XeonMations marked this pull request as draft December 20, 2023 21:52
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

To do:
On initialization, make new ID for the RBMK and have computers auto-locate and set their own IDs to it
Create and make the crate use the RBMK machine board instead of the machine + add the ability to move the RBMK when inactive
now onto the rest of the TODO list...
2
-Added the ability for the RBMK to alert engineers of its meltdown status over comms. (TODO: Add in a cooldown period)
-Fixed some bugs revolving around the computers not linking with RBMK
-Fixed bug where RBMK would not connect to a pipe layer
To do:

New sprites for the RBMK, include five fuel rod variations + show where pipes input and output
New SFX for the RBMK.
Get the three input pipes to be made automatically correctly.
Get the RBMK to be wrenchable and movable
@MarkusLarsson421
Copy link
Contributor

We don't want to use this in place of the supermatter crystal which leaves it in a place where its going to be unmaintained and never used.

If it is hidden in some corner of the game, it will be unmaintained because no one will ever go to that corner and know of its existence. I would say we SHOULD allow the SM to be replaced with other engines if the situation desires it. Because then people will actually care for it and its functionality further into the feature.

With how hidden Derelict Stations are, it is no wonder how unmaintained it is. Not because PRs are being shut down, but because no one knows of its existence to make the PRs to begin with.

@PowerfulBacon
Copy link
Member

A new mapping standard won't be set for stations to have their SM replaced with RBMK. We will suffer from having to maintain multiple engines, and the RBMK is not an appropriate option for a full SM replacement.

@Rukofamicom
Copy link
Contributor

I really love the RBMK, but I ultimately agree with Bacon.

I don't want to replace the supermatter crystals with it, and I don't believe this should be merged unless it is replacing them.

@Isaacnml
Copy link

Isaacnml commented Sep 24, 2024

Please dont kill it, from initial thought it was supposed to replace the defunct antimatter reactor, which it does a pretty good job, never meant to replace the SM in any way and engineering desperately needs new content to well.. play around gameplay loop currently is really vague

It's a replacement incase the SM explodes, not a full replacement for the SM

No clue where Ambic got the full replacement for the SM idea, it was always supposed and designed to be a secondary engine, not a main one

@MarkusLarsson421
Copy link
Contributor

Almost a year in development and now the close keys are coming in? Jesus Christ. A year of Xeons life wasted.

@XeonMations
Copy link
Contributor Author

A year of Xeons life wasted

I dont mind to be fair. I code as a hobby, something I do for fun. Not for a living.

@XeonMations
Copy link
Contributor Author

from initial thought it was supposed to replace the defunct antimatter reactor

But yes, the idea from the start was as a replacement for the now gone antimatter reactor.

As of current, if the SM explodes, the station only has solars as a power source unless they want to take a rather long time setting up a black hole (good luck with no power) or a tesla (Again, good luck without power).

Or engineering could just set up a second SM in the form of a shard, but I'm unsure if they'd put that much effort in before the shuttle is called.

@EvilDragonfiend
Copy link
Member

I am honestly fine with this, but I suppose we might need a kind of mapping tool to replace (or some ingame stuff to make a dedicated area quickly). Constructing a new engine for a whole round was tedious.

@XeonMations
Copy link
Contributor Author

we might need a kind of mapping tool to replace (or some ingame stuff to make a dedicated area quickly). Constructing a new engine for a whole round was tedious.

Entirely fair, if wished, i am able to add one.

@EvilDragonfiend
Copy link
Member

we might need a kind of mapping tool to replace (or some ingame stuff to make a dedicated area quickly). Constructing a new engine for a whole round was tedious.

Entirely fair, if wished, i am able to add one.

not right now. we are in a mapping disaster rn.

@ClownMoff

This comment was marked as off-topic.

Copy link
Member

@PowerfulBacon PowerfulBacon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eh whatever, this can replace the antimatter generator.

Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace arbitrary names like R or N. It tells no context.

Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks alright as long as change requests are applied.

code/modules/power/rbmk/rbmk_monitoring.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_monitoring.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_procs.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_procs.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_procs.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_monitoring.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_parts.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_parts.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/fuel_rods.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/fuel_rods.dm Outdated Show resolved Hide resolved
Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be a final.

code/modules/power/rbmk/rbmk_monitoring.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_monitoring.dm Outdated Show resolved Hide resolved
code/modules/power/rbmk/rbmk_procs.dm Outdated Show resolved Hide resolved
Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay, we'll see how this goes.

@EvilDragonfiend EvilDragonfiend added this pull request to the merge queue Nov 23, 2024
Merged via the queue into BeeStation:master with commit 0926e7d Nov 23, 2024
21 checks passed
@XeonMations XeonMations deleted the RBMK2ElectricBoogaloo branch November 23, 2024 21:11
@BarteG44
Copy link
Contributor

YES YES YES YES

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔑 Close Key (1/3) Indicates that someone has requested this PR to be keyed 🔑 Close Key (2/3) Feature Needs Testmerge Sound Sprites TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.